Added correct docker and terraform configuration command#53
Conversation
kiranchavala
force-pushed
the
kiran_terraform_doc2
branch
from
bf2e77c to
c7fdb45
Compare
| required_providers { | ||
| cloudstack = { | ||
| source = "cloudstack/cloudstack" | ||
| source = "localdomain/provider/cloudstack" |
There was a problem hiding this comment.
The previous one worked when I was testing locally @kiranchavala , may you want to add what localdomain/provider means ?
There was a problem hiding this comment.
after building the terraform provider locally using "make build", the generated go binary will contain the new code changes
To test the changes we have to use
source = "localdomain/provider/cloudstack
If we use " source "cloudstack/cloudstack" ,
when "terraform init" is called terraform will download the binary from terraform registry https://registry.terraform.io/providers/cloudstack/cloudstack/latest
and place it /.terraform.d/plugins/registry.terraform.io/cloudstack/cloudstack/0.4.0/linux_amd64/terraform-provider-cloudstack_v0.4.0/
If replace the binary with the binary we had built in the same path and do a "terraform apply" , you will encounter a checksum error as terraform.lock.hcl will be different
There was a problem hiding this comment.
thanks @kiranchavala I used to delete the lock file to test my changes, so that way it worked.
|
@rohityadavcloud @davidjumani @shwstppr Could you please review this |
Co-authored-by: David Jumani <dj.davidjumani1994@gmail.com>
Co-authored-by: David Jumani <dj.davidjumani1994@gmail.com>
|
If there are any new changes @kiranchavala pl submit as separate PR cc @harikrishna-patnala |
* Added correct docker and terraform configuration commands * Update README.md Co-authored-by: David Jumani <dj.davidjumani1994@gmail.com> * Update README.md Co-authored-by: David Jumani <dj.davidjumani1994@gmail.com> --------- Co-authored-by: Rohit Yadav <rohityadav89@gmail.com> Co-authored-by: David Jumani <dj.davidjumani1994@gmail.com>
4 participants
No description provided.